feat(U-Boot): add QoS configuration section by bryanbrattlof · Pull Request #582 · TexasInstruments/processor-sdk-doc

bryanbrattlof · 2025-12-12T17:10:58Z

Add basic documentation on how the CBASS QoS blocks function and how Sysconfig can be used to generate the files for U-Boot to apply during the SoC's bootup phase.

github-actions · 2025-12-12T17:11:35Z

New issues found with check_toc_txt.py:

WARNING:root:Invalid slug: configs/AM62AX/AM62AX_linux_toc.txt:47 inux/Foundational_Components/U-Boot/UG-QoS

github-actions · 2025-12-12T17:11:55Z

New warnings found with rstcheck:

source/linux/Foundational_Components/U-Boot/UG-QoS.rst:49: (ERROR/3) Unknown target name: "an excellent guide".

StaticRocket

Vale comments about simple language an passive voice should probably be addressed

configs/J784S4/J784S4_linux_toc.txt

github-actions · 2025-12-15T22:46:48Z

New issues found with check_toc_txt.py:

WARNING:root:Invalid slug: configs/AM62AX/AM62AX_linux_toc.txt:47 inux/Foundational_Components/U-Boot/UG-QoS

bryanbrattlof · 2025-12-16T00:04:47Z

I took care of most of the Vale issues. What I scanned seems to be stuff I don't think we can fix unless I skipped over something.

I also split up the file endings to a separate patch

source/linux/Foundational_Components/U-Boot/UG-QoS.rst

StaticRocket

There are still some lengthy sentences here that vale doesn't like. Going to let others voice opinions about that

Simple invisible character fixes Signed-off-by: Bryan Brattlof <bb@ti.com>

Add basic documentation on how the CBASS QoS blocks function and how Sysconfig can be used to generate the files for U-Boot to apply during the SoC's bootup phase. Co-authored-by: Randolph Sapp <res.sapp@gmail.com> Signed-off-by: Bryan Brattlof <bb@ti.com>

praneethbajjuri · 2026-02-04T21:14:09Z

configs/J721S2/J721S2_linux_toc.txt

 linux/Documentation_Tarball

-linux/Demo_User_Guides/Chromium_Browser
+linux/Demo_User_Guides/Chromium_Browser


Doesnt belong in this patch.

what patch are you referring? Do I need a separate PR to fix the white-space with this file before I update it?

StaticRocket

Typos and active voice

StaticRocket · 2026-02-11T20:27:49Z

source/linux/Foundational_Components/U-Boot/UG-QoS.rst

+======================
+
+Most transactions default to the lowest priority (ASEL 0, Order ID 0).
+During boot-up, `U-Boot can update`_ these settings using data from the.


Suggested change

During boot-up, `U-Boot can update`_ these settings using data from the.

During boot-up, `U-Boot can update`_ these settings with data from the

StaticRocket · 2026-02-11T20:28:46Z

source/linux/Foundational_Components/U-Boot/UG-QoS.rst

+
+Most transactions default to the lowest priority (ASEL 0, Order ID 0).
+During boot-up, `U-Boot can update`_ these settings using data from the.
+Sysconfig Tool which you can download or launch online `here`_.


Suggested change

Sysconfig Tool which you can download or launch online `here`_.

Sysconfig Tool, which you can download or use online `here`_.

StaticRocket · 2026-02-11T20:29:08Z

source/linux/Foundational_Components/U-Boot/UG-QoS.rst

+
+The Common Bus Architecture (CBASS) module includes Quality of Service
+(QoS) blocks to route and prioritize SoC bus traffic. By adjusting
+attributes like priority, Address Selection (ASEL), and Order ID


Suggested change

attributes like priority, Address Selection (ASEL), and Order ID

attributes such as priority, Address Selection (ASEL), and Order ID

StaticRocket · 2026-02-11T20:33:43Z

source/linux/Foundational_Components/U-Boot/UG-QoS.rst

+.. _U-Boot can update: https://source.denx.de/u-boot/u-boot/-/blob/v2025.10/arch/arm/mach-k3/am62px/am62p5_init.c?ref_type=tags#L253
+.. _here: https://www.ti.com/tool/SYSCONFIG
+
+The MCU+ SDK documentation has `an excellent guide`_ on how to to use the


Suggested change

The MCU+ SDK documentation has `an excellent guide`_ on how to to use the

The MCU+ SDK documentation has `an excellent guide`_ on how to use the

StaticRocket · 2026-02-11T20:34:52Z

source/linux/Foundational_Components/U-Boot/UG-QoS.rst

+.. note::
+
+   Configuring the QoS blocks of a running system can cause issues.
+   You can only modify these settings during boot-up by the boot-loaders


Suggested change

You can only modify these settings during boot-up by the boot-loaders

You can only change these settings during boot by using the boot-loaders,

StaticRocket · 2026-02-11T20:36:54Z

source/linux/Foundational_Components/U-Boot/UG-QoS.rst

+
+For example, most K3 SoC External Memory Interface (EMIF) controllers
+use two ports. Setting an Order ID between 8 and 15 routes traffic
+through a high-priority port, ensuring it is serviced before standard


Suggested change

through a high-priority port, ensuring it is serviced before standard

through a high-priority port, ensuring it gets serviced before standard

bryanbrattlof · 2026-02-11T20:48:06Z

No. I've had enough. This has taken me 2 months to get through the nitpicking. So long in fact the people originally asking for this documentation are no longer with TI. I will be marking this a blocked until we get a proper copy editor.

I have better things to do with my time.

StaticRocket · 2026-02-13T23:35:19Z

Going to be real, you could have at least read the linter warnings 2 months ago. These nitpicks are suggestions to the same linter warnings you've had since the start of this PR.

bryanbrattlof requested review from StaticRocket, VeeruPrudhvi, cshilwant, gehariprasath, jeevantelukula, praneethbajjuri and uditkumarti as code owners December 12, 2025 17:11

github-actions bot assigned bryanbrattlof, devarsht, gehariprasath, jmenti, praneethbajjuri, r-vignesh and uditkumarti Dec 12, 2025

bryanbrattlof marked this pull request as draft December 12, 2025 21:13

bryanbrattlof force-pushed the sdk11.2-qos branch 3 times, most recently from 792589f to c2884e4 Compare December 12, 2025 21:47

bryanbrattlof marked this pull request as ready for review December 12, 2025 22:05

StaticRocket requested changes Dec 13, 2025

View reviewed changes

configs/J784S4/J784S4_linux_toc.txt Show resolved Hide resolved

bryanbrattlof marked this pull request as draft December 15, 2025 13:09

bryanbrattlof force-pushed the sdk11.2-qos branch from c2884e4 to 7995b38 Compare December 15, 2025 22:46

bryanbrattlof force-pushed the sdk11.2-qos branch 3 times, most recently from a17be0c to 12fdf23 Compare December 15, 2025 23:59

bryanbrattlof marked this pull request as ready for review December 16, 2025 00:03

StaticRocket requested changes Dec 16, 2025

View reviewed changes

bryanbrattlof force-pushed the sdk11.2-qos branch from 8c0767a to 60ec232 Compare December 16, 2025 13:32

StaticRocket reviewed Dec 16, 2025

View reviewed changes

cshilwant added the stale PR is not updated by Author in recent times. Will be declined after 1 week label Jan 25, 2026

bryanbrattlof and others added 2 commits February 3, 2026 11:19

feat(fix): ensure files end with a newline

59704bf

Simple invisible character fixes Signed-off-by: Bryan Brattlof <bb@ti.com>

bryanbrattlof force-pushed the sdk11.2-qos branch from 60ec232 to 06cc12a Compare February 3, 2026 17:30

praneethbajjuri reviewed Feb 4, 2026

View reviewed changes

StaticRocket removed the stale PR is not updated by Author in recent times. Will be declined after 1 week label Feb 5, 2026

bryanbrattlof requested review from StaticRocket and praneethbajjuri February 10, 2026 14:52

StaticRocket requested changes Feb 11, 2026

View reviewed changes

bryanbrattlof closed this Feb 11, 2026

	During boot-up, `U-Boot can update`_ these settings using data from the.
	During boot-up, `U-Boot can update`_ these settings with data from the

	Sysconfig Tool which you can download or launch online `here`_.
	Sysconfig Tool, which you can download or use online `here`_.

	attributes like priority, Address Selection (ASEL), and Order ID
	attributes such as priority, Address Selection (ASEL), and Order ID

	The MCU+ SDK documentation has `an excellent guide`_ on how to to use the
	The MCU+ SDK documentation has `an excellent guide`_ on how to use the

	You can only modify these settings during boot-up by the boot-loaders
	You can only change these settings during boot by using the boot-loaders,

	through a high-priority port, ensuring it is serviced before standard
	through a high-priority port, ensuring it gets serviced before standard

Conversation

bryanbrattlof commented Dec 12, 2025

Uh oh!

github-actions bot commented Dec 12, 2025

Uh oh!

github-actions bot commented Dec 12, 2025

Uh oh!

StaticRocket left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

github-actions bot commented Dec 15, 2025

Uh oh!

bryanbrattlof commented Dec 16, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

StaticRocket left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

StaticRocket left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

bryanbrattlof commented Feb 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

StaticRocket commented Feb 13, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants

bryanbrattlof commented Feb 11, 2026 •

edited

Loading